-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(deep-links): Fix link to pkg.go.dev to not return 404 #14595
Conversation
Signed-off-by: Håkon Solbjørg <[email protected]>
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## master #14595 +/- ##
=======================================
Coverage 49.81% 49.82%
=======================================
Files 261 261
Lines 44796 44796
=======================================
+ Hits 22317 22320 +3
+ Misses 20280 20278 -2
+ Partials 2199 2198 -1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @sklirg !! LGTM!!
Anything else you need from me, @ishitasequeira ? :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @sklirg!
/cherry-pick release-2.8 |
/cherry-pick release-2.7 |
/cherry-pick release-2.6 |
Signed-off-by: Håkon Solbjørg <[email protected]>
Signed-off-by: Håkon Solbjørg <[email protected]>
Cherry-pick failed with |
…14640) Signed-off-by: Håkon Solbjørg <[email protected]> Co-authored-by: Håkon Solbjørg <[email protected]>
…14639) Signed-off-by: Håkon Solbjørg <[email protected]> Co-authored-by: Håkon Solbjørg <[email protected]>
…14595) (argoproj#14640) Signed-off-by: Håkon Solbjørg <[email protected]> Co-authored-by: Håkon Solbjørg <[email protected]> Signed-off-by: schakrad <[email protected]>
…14595) Signed-off-by: Håkon Solbjørg <[email protected]>
…14595) Signed-off-by: Håkon Solbjørg <[email protected]> Signed-off-by: Jimmy Neville <[email protected]>
…14595) Signed-off-by: Håkon Solbjørg <[email protected]>
Right now, the link returns a 404 because it is redirected to
https://argo-cd.readthedocs.io/en/stable/operator-manual/deep_links/pkg.go.dev/text/template
.Steps to reproduce:
https://argo-cd.readthedocs.io/en/stable/operator-manual/deep_links/pkg.go.dev/text/template
which is a 404 (if you scroll all the way down).Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
[ ] I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.[ ] I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.[ ] Does this PR require documentation updates?[ ] Optional. My organization is added to USERS.md.[ ] I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.[ ] My new feature complies with the feature status guidelines.Please see Contribution FAQs if you have questions about your pull-request.